cell renderer spin: Drop the priv pointer

cell renderer spin: Make final

cell renderer spinner: Drop priv pointer

cell renderer spinner: Make final

cell renderer text: Drop the priv pointer

cell renderer text: Use standard padding

cell renderer toggle: Drop the priv pointer

cell renderer toggle: Make final

cell renderer combo: Drop the priv pointer

cell renderer combo: Make final

cell renderer accel: Drop the priv pointer

cell renderer accel: Make final

cell renderer: Use standard padding

calendar: Drop the priv pointer

calendar: Make final

spinner: Make final

progress bar: Make final

print operation: Drop the priv pointer

print operation: Use standard class padding

Rename gtkprintbackend.h to gtkprintbackendprivate.h

Follow the naming convention for private headers.

print job: Make final

No need to subclass this.

printer: Drop the priv pointer

printer: Make private

We want to subclass this in print backends,
but we don't want to allow out-of-tree subclassing.

Rename gtkprinter-private.h

Name this header gtkprinterprivate.h, following
our convention for private headers.

stack switcher: Make final

This should not be subclassed.

stack sidebar: Make final

This should not be subclassed.

print dialog: Drop the priv pointer

print dialog: Make final

Nobody should subclass this.

page setup dialog: Drop the priv pointer

page setup dialog: Make final

No subclassing needed.

switch: Make final

This should not be subclassed.

icon theme: Make final

Nobody should subclass GtkIconTheme or GtkIconInfo.

textview: Use standard padding

Given that GtkSourceView exists, we will have
to keep GtkTextView subclassable until a replacement
appears.

viewport: Cleanup

viewport: Make final

No need to derive from it.

Trivial

surface: fix up docs

The docs for gdk_surface_invalidate_region were
talking about a no-longer-existing argument.

color plane: Don't allocate 0-size textures

This triggers critials, and does us no good.

Cosmetics

rendernodeparser: Parse Cairo nodes

We also print the script, but we don't parse it yet.

rendernode: Remove unused function

inspector: Make updates overlay not crash

... when a window gets hidden and later reshown.

The code now properly cleans up a window when it gets unmapped instead
of trying to retain previous updates information.

Revert "wayland: Keep some resources until destroy"

This reverts commit f2d598b9a19cd7151c592e9c74d5b0d0ba3c9e48.

widget-factory: Add some animation in a popover

broadway: add frame times to the profiler

fishbowl: Add a menubutton demo

This reveals some issues with the position tracking.

menu button: Add popup/popdown functions

These are needed, after all.

fishbowl: Fix the lock button

gl renderer: Make creating render targets easier

gl renderer: Remove some unused api form the GLDriver

fishbowl: Don't show ridiculous precision

Two decimals are more than enough for the frame rate.

Merge branch 'issue1901-gtk4' into 'master'

wayland: Use scale from core wl_output protocol

Closes #1901

See merge request GNOME/gtk!870

wayland: Use scale from core wl_output protocol

The “xdg-output” protocol provides clients with the outputs size and
position in compositor coordinates, and does not provide the output
scale which is already provided by the core “wl_output” protocol.

So when receiving the wl_output scale event, we should update the scale
regardless of “xdg-output” support, otherwise the scale will remain to
its default value of 1 and the surface will be scaled up by the
compositor to match the actual output scale, which causes blurry fonts
and widgets.

Fixes: https://gitlab.gnome.org/GNOME/gtk/issues/1901
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

gtk-demo: Reinstate flat menu buttons

widget-factory: Reinstate flat menu buttons

menu button: Add a relief property

This is used in various places.

testscrolledwindow: GtkMenuButton != GtkContainer

It's not anymore, so don't cast it to one... Use the new :label instead.

testsuite: Fix leak

Merge branch 'mkenums-once' into 'master'

Use g_once for all the mkenums template files

Closes #1617

See merge request GNOME/gtk!865

Use g_once for all the mkenums template files

Without this, initialization of enum/flags values from multithreaded
programs may not work correctly.

Fixes #1617.

Merge branch 'win32-enable-werror' into 'master'

Win32 enable werror

See merge request GNOME/gtk!861

Merge branch 'issue-302-master' into 'master'

docs: Fix the override for GdkAtom

Closes #302

See merge request GNOME/gtk!864

docs: Fix the override for GdkAtom

GdkAtom is a typedef to a pointer to an opaque structure. We need to
tell GTK-Doc how to override it, so that the documentation is accurate.

Fixes: #302

gl renderer: Render recording surface untransformed

Cairo can't handle the y flip apparently.

Fixes #1906

gl glyphcache: Don't choke on huge glyphs

We blindly assume everywhere that a single glyph will definitely fit on
one atlas, but that's not always the case.
For now, don't crash or produce GL errors.

widget: Don't always call get_display() when allocating

The GTK_DISPLAY_DEBUG_CHECK macro will cleverly only call the function
if any of the display debug flags are set, so in the common case it
won't even be executed.

win32: Fix meson names for defines

Those have been wrong forever, but nobody cares because they get to the
values we want anyway.

win32: Enable --werror in CI

win32: Remove outdated workaround

The function that was missing in 2016 does exist now.

Fixes gcc complaining about a wrong declaration.

tests: Use g_free(), not free()

Windows doesn't like that.

imcontextime: Add missing semicolon

I wonder how thoroughly this was tested... ;)

win32: Add missing enum value to switch statement

imcontextime: Remove unused variables

win32: Rename a shadowing variable

window: Remove unneeded variable

The variable would show up as a gcc warning in builds without X11
support.

win32: Actually pass y variable for y parameter

win32: Remove unused variables

Merge branch 'wip/otte/for-master' into 'master'

Wip/otte/for master

See merge request GNOME/gtk!860

testsuite: Remove serialize-deserialize test

The serialize-deserialize test is completely handled via the new
nodeparser tests.

So move those tests over and delete the old test binary.

testsuite: Add render tests for all the empty nodes

Test that rendering empty nodes succees. For a lot of nodes the
resulting rendering isn't clearly defined, in those cases we overdraw
those regions (sometimes the whole image) with black.

testuite: Improve compare-render binary

- Remove remains of g_test_*() functions
  We're not a glib test, we're a simple binary.
- Handle nonexistence of reference image properly
  Don't assert, but create the output image and the error out.

rendernode: Blur alpha channel for cairo blur node

The blur node still doesn't properly blur with a transform, but at least
it now doesn't turn semi-transparent stuff black.

testsuite: Add empty node tests

For every node type, add a test that creates it empty and ensure that
it prints the reference output.

rendernodeparser: Allow parsing strings

Instead of only allowing for glyph indexes, allow ASCII characters as
replacements. So this glyph sequence
  glyphs: 65 8, 66 8, 67 8
Can be replaced by
  glyphs: "ABC"
provided that the glyph for "A", "B" and "C" are 65, 66 and 67
respectively and their advance is exactly 8.
x offset and y offset must always be 0 and every glyph must start a
cluster.

rendernodeparser: Update to new rules

Update to the docs outlined in #1887.

In particular, the changes do:

1. Require no property, have a working default for everything
2. Be clear about what gets printed and how.

Tests ahve been adapted to still pass.

rgba: Add GDK_RGBA() macro

So far it's private, but it's a pretty cute way to use hex colors, so we
might conside making it public.

rendernode: Take a graphene_point_t for the offset

... instead of 2 floats.

testsuite: Redo node-parser

Base the rewrite on testsuite/css/parser/test-css-parser - we now
require the node file to match a reference node and track the errors it
triggers.
We also no longer use gtester.

rendernodeparser: Skip root node when it's a container

When printing, behave the same way as when parsing:
Magically skip a container node if there is one - just like the
parser magically creates a container node to hold all the nodes
it parses.

rendernodeparser: Allow single values instead of 4

This allows writing:
  colors: red;
instead of
  colors: red red red red;
to draw a red border.

rendernodeparser: Print properties alphabetically

Because now we have a rule.

Also update some tests to the new order.

glrenderer: Handle NULL debug messages

gtk-demo: Fix menubutton usage in listbox demo

Menu buttons are no longer buttons, so they
can't have children, and don't have a relief
property.

inspector: Don't apply button api to menu buttons

gl renderer: Pass correct matrix category when rendering offscreen

We can't just pretend we have an identity matrix when we are actually
scaling. This fixes the node editor sometimes not drawing things when
rendering to a texture. We were mistakenly discaring render nodes
because the bounds transformation was wrong.

gl renderer: Delete render_texture framebuffer

gl renderer: A GdkGLTexture's context might be NULL

gdk_gl_texture_download ought to still work, but we can't make a NULL
context current obviously.

gl renderer: Really fix gl debugging code

Properly label an object and make sure we have the right gl context even
after a gdk_gl_context_end_frame call.

builder-tool: Remove debug spew

x11: Remove to unused settings

We no longer have settings for gtk-button-images
or gtk-menu-images.